Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade Silicon Labs radio driver to v2.1.1 #5579

Merged
merged 4 commits into from Nov 30, 2017

Conversation

stevew817
Copy link
Contributor

Description

Upgrades the RAIL library (on which the Nanostack driver is based) from v1.5.x to v2.1.1. The upgrade from 1.x to 2.x includes a whole lot of API changes, but in the end, makes RAIL more efficiently run alongside an RTOS.

Additional improvements:

  • Stubs for all events
  • Binary should now be fully AAPCS-compliant (no more ambiguity on enum size), so should work across toolchains
  • Added EFR32_x flags to remove hard dependency on Mighty Gecko parts (should work on BG and FG parts too, even though they don't have mbed-enabled development kits).
  • Lays the foundation for potential BLE enablement on EFR32

Status

READY

Migrations

No behavioral change intended, no external API changes.

@stevew817
Copy link
Contributor Author

Also resolves #5382

Need to avoid a TX packet error from messing up the TX FIFO. Since this API will return busy if a packet is in transmission anyways, this is acceptable.
@stevew817
Copy link
Contributor Author

CC @screamerbg @ashok-rao @ryankurte

@0xc0170
Copy link
Contributor

0xc0170 commented Nov 29, 2017

/morph build

@mbed-ci
Copy link

mbed-ci commented Nov 29, 2017

Build : SUCCESS

Build number : 618
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/5579/

Triggering tests

/morph test
/morph uvisor-test
/morph export-build

@0xc0170
Copy link
Contributor

0xc0170 commented Nov 29, 2017

@orenc17A Please review the failure for uvisor CI. Seems like target interface related

@mbed-ci
Copy link

mbed-ci commented Nov 29, 2017

@mbed-ci
Copy link

mbed-ci commented Nov 29, 2017

@stevew817
Copy link
Contributor Author

@0xc0170 Seems the failure is gone now?

@ryankurte
Copy link
Contributor

ryankurte commented Dec 18, 2017

@stevew817 is there a reason this doesn't include the ieee802154_subg_efr32xg12_configurator_out source/header files? (And are these generated from the default options in the configurator?)

-edit s\efr32xg1\efr32xg12\g-

@ryankurte
Copy link
Contributor

Follow up: this seems to need some modification to get it working.

With a quick-and-dirty copy of the configurator files the updated version results in a Thread 0x0 error -4: Parameter error which appears to originate somewhere in the depths of rf_device_register function. I can not however debug this because of the rail blob.

Shall I open another issue against this?

@stevew817
Copy link
Contributor Author

@ryankurte Yes, please log a separate issue. Not sure why they disappeared, but I'll look into it.

@SeppoTakalo
Copy link
Contributor

This Thread error might be related to #5680 and #5587

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants